-
Notifications
You must be signed in to change notification settings - Fork 321
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[stable-2.7] west.yml: update Zephyr to sofproject stable-v2.7 #8353
[stable-2.7] west.yml: update Zephyr to sofproject stable-v2.7 #8353
Conversation
SOFCI TEST |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At least the FW still builds with this PR after west update, and the back-ported Peter's DMA fix commit is on top of Zephyr repository.
@kv2019i I assume we are expecting the CI failures if CI is expecting to target config from main branch ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kv2019i I assume we are expecting the CI failures if CI is expecting to target config from main branch ?
Yes and no.
- Yes the "zmain" builds are expected to fail because there's not a chance SOF stable-v2.7 compiles with the fast moving Zephyr main branch. Good news: I just removed that combination in #8368 (@dbaluta you want to cherry-pick -x
that to imx-stable-v2.7)
- No the manifest (mnfst
) builds are NOT expected to fail! They did not fail at all in recent stable-v2.7 PRs #8225 and #8368. This is happening only with this PR, so there's something wrong with it. Looking into it.
https://github.com/thesofproject/sof/actions/runs/6576490892/job/17866031229?pr=8353
fatal: 'zephyrproject' does not appear to be a git repository
Fixes issue reported in stable-v2.7 thesofproject#8353 which wants to use our own, downstream Zephyr branch. Signed-off-by: Marc Herbert <[email protected]>
Sparse failure in https://github.com/thesofproject/sof/actions/runs/6576490890/job/17866025157?pr=8353. I suspect it's caused the only commit (so far) on this branch from @ujfalusi : 3c58926487ebef001e. It looks like this cherry-pick was pushed without any PR! EDIT: I'm an idiot. This is exactly what this PR does. Fix submitted in:
|
335c51b
to
0e61eb3
Compare
Fixes issue reported in stable-v2.7 #8353 which wants to use our own, downstream Zephyr branch. Signed-off-by: Marc Herbert <[email protected]>
0e61eb3
to
866d0b3
Compare
Fixes issue reported in stable-v2.7 thesofproject#8353 which wants to use our own, downstream Zephyr branch. Signed-off-by: Marc Herbert <[email protected]> (cherry picked from commit 4bc6488) Signed-off-by: Kai Vehmanen <[email protected]>
GitHub conveniently times each step but to reduce step proliferation we sometimes have multiple commands in a single step and lose the ability to time download commands. Prefix them with `time` to restore the data. Signed-off-by: Marc Herbert <[email protected]> (cherry picked from commit 92fa795) Signed-off-by: Kai Vehmanen <[email protected]>
Update Zephyr to use thesofproject/zephyr/stable-v2.7 with following commits backported from Zephyr main. 3c58926487eb drivers: dma: intel-adsp-hda: Correct DGCS:SCS bit for feaa6281b7fc drivers: hda: insert an empty ";" statement before Link: thesofproject#8236 Signed-off-by: Kai Vehmanen <[email protected]>
Set version to 2.7.1 as we start merging bugfixes. Signed-off-by: Kai Vehmanen <[email protected]>
Screwed up the N-1 update, but latest one is now good. The windows build tests fail, but I believe this is something we fixed in main, right? Otherwise this should be good to go now. |
Only now, sorry I missed Windows duplication the first time: |
@marc-hb @wszypelt The TGL QB tests are failing due to rimage option problem. The resulting binary is a TGL IPC4 binary. but rimage uses IPC3 toml config, so resulting binary is not valid and will not boot. I'll proceed to merge this PR, and will submit a follow-up PR to enable testing TGL on this stable branch as well. |
Fixes issue reported in stable-v2.7 #8353 which wants to use our own, downstream Zephyr branch. Signed-off-by: Marc Herbert <[email protected]> (cherry picked from commit 4bc6488) Signed-off-by: Kai Vehmanen <[email protected]>
commit 335c51b (HEAD -> stable-v2.7)
Author: Kai Vehmanen [email protected]
Date: Thu Oct 19 17:59:52 2023 +0300
commit f6b74af
Author: Kai Vehmanen [email protected]
Date: Thu Oct 19 17:56:49 2023 +0300